Skip to content

fix: prepare planner payloads from ai review artifacts - #19

Merged
Pigbibi merged 1 commit into
mainfrom
fix/planner-build-payloads
Apr 1, 2026
Merged

fix: prepare planner payloads from ai review artifacts#19
Pigbibi merged 1 commit into
mainfrom
fix/planner-build-payloads

Conversation

@Pigbibi

@Pigbibi Pigbibi commented Apr 1, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • make the planner rebuild normalized review payloads from issue_context.json and secondary_review.json after artifact download
  • keep the planner compatible with both old dual-AI runs and new runs that already include final_review_payload.json
  • update workflow config tests for the new payload preparation steps

Testing

  • python3 -m unittest tests/test_build_ai_review_payload.py tests/test_download_ai_review_artifact.py tests/test_build_monthly_optimization_plan.py tests/test_post_monthly_optimization_issue.py tests/test_post_monthly_ai_review_comment.py tests/test_render_monthly_ai_review.py tests/test_run_openai_secondary_review.py tests/test_monthly_publish_workflow_config.py tests/test_monthly_optimization_planner_workflow_config.py
  • uvx ruff check scripts tests
  • ruby -e 'require "yaml"; Dir[".github/workflows/*.yml"].each { |f| YAML.load_file(f) }; puts "yaml ok"'

@Pigbibi
Pigbibi merged commit 76222da into main Apr 1, 2026
1 check passed
@Pigbibi
Pigbibi deleted the fix/planner-build-payloads branch April 11, 2026 18:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant